Add opencl support #2176
Closed
Add opencl support #2176
+75
−8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Installing libOpenCL.so.1 will clash with links inhere. - remove libOpenCL.so.1 - remove duplicate libnvidia-opencl.so.1 (?) - remove remaining unreferenced links Note: libnvidia-wfb.so.1 and libOpenCL.so.1 _are_ part of the nvidia-390.87 driver, but no workstation uses this driver anymore, so drop this cruft. The tool for checking is named check-nvidia-mxlinks.sh, and can be found in /project/admin/tools
Minimum required to build and run OpenCL Apps
|
bee shows a conflict: |
|
That's the situation that nvidia-mxlinks-0.12-1 is about to resolve.
It looks like something with nvidia-mxlinks went really haywire.
There is a nvidia-mxlinks-0.13-0 in /src/mariux/beeroot/bee-files dated
Jan-2020, that looks way experimental to me. OTOH nvidia-mxlinks-0.12-0
in the repo doesn't even 'build'. (duplicate libnvidia-opencl.so.1 ...)
To proceed with testing, nvidia-mxlinks-0.12-1 could be used, but to
resolve this situation. I simply propose to bump up the version high
enough and avoid further confusion.
|
|
Thank you for the analysis. I didn’t check the version numbers. So how to test this properly? |
pmenzel
reviewed
Aug 17, 2021
| # SRCURL[0]="https://github.com/OCL-dev/ocl-icd/archive/refs/tags/v${PKGVERSION}.tar.gz" | ||
| SRCURL[0]="https://beehive.molgen.mpg.de/7c34ddef4a662f6737e3165ccd417cba/ocl-icd-2.3.0.tar.gz" | ||
|
|
||
| # PATCHURL+=() |
There was a problem hiding this comment.
The comment can be removed?
pmenzel
reviewed
Aug 17, 2021
| SRCURL[0]="https://beehive.molgen.mpg.de/7c34ddef4a662f6737e3165ccd417cba/ocl-icd-2.3.0.tar.gz" | ||
|
|
||
| # PATCHURL+=() | ||
| PATCHURL[0]="https://beehive.molgen.mpg.de/a2de5a61942a5360560f415cfe0cbd61/0001-Updated-to-support-latest-Khronos-headers.patch.gz" |
There was a problem hiding this comment.
It’d be great if you mentioned in the commit message, where the patch is from.
So I suggest to either replace |
|
So how to test this properly?
Try some OpenCL software, e.g. clinfo should work now.
|
|
I'd like to keep the comment, it points to the origin.
|
Sorry, I just meant, what bee packages to update and to install (and what order). But yes, I tested with clinfo, and it worked. |
|
a2de5a6 is from the OCL-dev/ocl-icd repo (see prev message). But right,
it should be mentioned.
|
|
In the diff I wondered also about 'libGLX_indirect.so.0', it rings a
bell, but there is no trace in the current system (AFAICT)
So I suggest to either replace |nvidia-mxlinks-0.13-0|, or use 0.14-0.
Jepp, that was the point with 'bumping up the version'.
…--
Thomas Kreitler - Information Retrieval
kreitler@molgen.mpg.de
49/30/8413 1702
|
|
Sorry, I just meant, what bee packages to update and to install (and
what order). But yes, I tested with clinfo, and it worked.
Just a few steps, install cleaned up nvidia-mxlinks, install recent
CL-headers (no rebuild required), and ocl-icd at the end.
Basically the libOpenCL.so.1 conflict is just an annoyance, since it was
already a dead link when running newer nvidia-drivers. After ocl-icd
installation the link is correct.
|
|
#2180 obsoletes this one, also the chaos around nvidia-mxlinks distracts from the real aim of the PR. So I'm closing this one before it gets really ridiculous. |
Sign in
to join this conversation on GitHub.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Can be tested with compiling and running cas-offinder (https://github.com/snugel/cas-offinder).